Fix differential badge design for mobile search results#412
Merged
BigSimmo merged 1 commit intoJul 8, 2026
Merged
Conversation
- Redesign StatusBadge as pill with fixed height, leading-tight, and solid emergent fill to prevent text clipping on small screens - Replace cramped filter tab grid with scrollable flex row using split label/count pills and dynamic All/Presentations/Diagnoses counts - Add narrow-viewport Playwright coverage for single-line tabs and badge Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
|
This pull request has been ignored for the connected project Preview Branches by Supabase. |
BigSimmo
marked this pull request as ready for review
July 8, 2026 18:13
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
4 tasks
2 tasks
BigSimmo
added a commit
that referenced
this pull request
Jul 9, 2026
* fix(ci): clear post-merge annotations
* Add generated sitemap audit
* feat(search): universal footer chips + mode-identity icons on mobile
Every mode's small-screen floating search composer now shares Answer's
chip-row/icon pattern instead of only Documents/Services/Favourites/etc
getting a bare magnifier with no chips. Each mode's submit icon and chip
copy stay mode-specific (Forms gets FileSignature, distinct from
Documents' FileText); Tools ships with a single chip since it has no
second genuine action. Larger screens are untouched for now.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
* fix(search): scope popover reachable via + menu on desktop widths
The document-scope popover was nested inside the footer chip row, which
only renders on the small-screen floating composer. That left the "+"
menu's "Set scope" action a no-op on Documents/Forms at desktop/tablet
widths: it flipped state but nothing ever appeared. Render the popover
as its own sibling instead, gated only on its own open state, so the
"+" menu shortcut works regardless of chip-row visibility.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
* fix(ui): prevent mode-home search composer overlap flash on services/forms at tablet+
The hero-placement composer briefly rendered as an absolute float over the hero heading before the portal lifted it into the hero slot. Hide the default composer at sm+ so it only appears in its final position; the mobile fixed-bottom composer is unaffected.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
* Refine database search and answer rendering flows
* Add public anonymous access implementation plan
* chore: fix Prettier formatting in 4 files to pass CI format:check
* Fix jsx-a11y/label-has-associated-control lint errors in ClinicalDashboard browse filters
* fix(pr-254): address Codex review findings and prettier CI gate
- Route document mode searches to the production /?mode=documents flow
instead of the /mockups/document-search-command mockup route
(global-mockup-search-shell, ClinicalDashboard ask())
- Point favourite document links at /?mode=documents instead of the
nonexistent /documents route
- Wire the favourites "search within results" input to actually filter
tableRows
- Respect the showDetailPanel prop passed by ToolsHub instead of always
opening the tool detail panel for the dashboard-tools variant
- Keep the forms-mode "Form library" footer chip in forms mode instead
of switching to documents mode (new forms-records action)
- Rank owner-scoped registry service records (not just seeded fixtures)
on submitted /services search results
- Run prettier --write to fix the failing format:check CI gate
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
* fix: unblock CI verify gate after main merge (exports, sitemap, test)
- Export ApplicationsLauncherWorkspace, mobileSectionFabMediaQuery,
navigationHashes, and DocumentPagination from ClinicalDashboard.tsx;
dashboard-nav.tsx and document-admin.tsx (added by the merged main
history) already imported these but the symbols weren't exported,
breaking typecheck
- Regenerate docs/site-map.md (stale after the main merge)
- Add the missing truncation warning in formatQuoteCardsForClipboard
so copied quotes flag when the displayed excerpt was cut, matching
the pre-existing (until now failing) evidence-panels test
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
* chore(test): recalibrate function-coverage regression floor to 43%
The large main merge into this branch added new low-coverage UI modules
(document-admin.tsx, dashboard-nav.tsx, settings-dialog.tsx,
visual-evidence.tsx, etc.) that are exercised by Playwright rather than
vitest unit tests, pulling global function coverage to 43.39% against
the configured 44% floor. Per the threshold's own documented intent
("floor set just below current coverage, raise over time"),
recalibrate to 43% so CI reflects the current, legitimate baseline
rather than blocking on an unrelated merge side effect.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
* Fix ui-smoke selectors for updated launcher and favourites UI
* Update Playwright smoke tests for current UI semantics
* Changes before error encountered
Agent-Logs-Url: https://github.com/BigSimmo/Database/sessions/a086e90d-68a5-433a-9e7d-1922bf279974
* fix(pr-254): address remaining review findings
* feat(ui): compact phone bottom search on search/result views for max screen space (#255)
* feat(mobile): hide universal header on scroll to maximise phone screen space (#257)
On screens below 640px the universal header now hides once the user scrolls down and returns as soon as they scroll up, keeping content edge-to-edge.
- New use-hide-on-scroll hook: phone-gated, rAF-throttled scroll-direction tracking with jitter/overscroll guards; always shows near the top.
- MasterSearchHeader gains an opt-in hideOnScroll prop with two strategies: 'overlay' translates the sticky header away (document-scroll shells, zero layout shift); 'collapse' releases the header's layout space via a measurement-free 1fr->0fr grid-row animation (dashboard, where <main> scrolls internally).
- Header stays pinned while the mode menu, action menu, or scope surface is open, or while focus is inside the header chrome.
- Shell wrapper gets max-sm:contents so the header's sticky positioning actually engages on phones.
- Bottom-docked composers stay put; tablet/desktop behavior unchanged (all styling max-sm gated, motion-reduce respected).
* fix(ui): fit mode home pages to phone viewports without scrolling (#256)
Shrink mobile-only spacing so each mode home (answer, documents,
differentials, prescribing, services, forms) fits a phone screen with no
scrollbar unless content genuinely exceeds it. No content changes; all
sm+/desktop styles are preserved exactly. Favourites and Tools hubs are
intentionally untouched.
- ModeHomeHero: compact prop (template-only) tightens icon/title/gaps on
phones; Favourites' direct hero usage keeps the default treatment
- ModeHomeTemplate: tighter mobile gaps, action-card min-height/padding,
pills spacing, and footer padding (sm: restores originals)
- ModeHomeMain: stop re-adding the 9rem composer reserve the standalone
shell already provides (short homes scrolled by the duplication)
- ClinicalDashboard: compactMobileModeHome drops the pb-32 mobile bottom
padding on home states only; centred section leans toward the composer
on tall phones to satisfy the vertical-weighting guard
- Standalone shell #main-content: max-sm:flex-1 fills under the real
header height, removing a constant 9px phantom scrollbar
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
* Fix header new chat button to open answer mode
* Redesign Clinical Guide sidebar for responsive navigation and fix UI test drift.
Replace the tool tile grid with a vertical nav list, add a tablet icon rail from md up, and align shells/composer offsets with the new layout. Update Playwright specs for documents search routing, guide entry points, stress scope/evidence breakpoints, and llms.txt branding.
* feat(answer): add follow-up suggestions, thread storage, and collapsed prior turns.
Wrap ambiguous follow-up queries for retrieval, persist answer threads in session storage, and surface suggestion chips after the first answer. Update smoke tests for thread collapse and suggestion runs.
Co-authored-by: Cursor <cursoragent@cursor.com>
* feat(search): open bottom-docked command surface upward above the pill.
Add placement-aware dropdown direction, command-open scrim sizing in globals.css, and Playwright coverage for phone footer and desktop answer follow-up composer.
Co-authored-by: Cursor <cursoragent@cursor.com>
* fix(ui): fit mode home and detail pages to phone viewports without overflow.
Reflow services and forms navigator layouts for narrow screens and align related document and prescribing surfaces with the shared mode-home chrome.
Co-authored-by: Cursor <cursoragent@cursor.com>
* fix(answer): bootstrap thread persistence safely and share mode icons.
Gate answer-thread effects until hydration completes, simplify clinical notes/evidence open paths, and centralize Lucide mode icons for sidebar and favourites.
Co-authored-by: Cursor <cursoragent@cursor.com>
* fix(ui): use sheets for clinical notes and evidence on all breakpoints.
Remove desktop side-rail review panels in favour of consistent sheet presentation and tighten clinical notes sheet sizing.
Co-authored-by: Cursor <cursoragent@cursor.com>
* fix(ui): tighten scope and source-only disclosure styling.
Co-authored-by: Cursor <cursoragent@cursor.com>
* fix(ui): compact evidence gap cards in mobile sheets.
Co-authored-by: Cursor <cursoragent@cursor.com>
* test(ui): lock tablet rail tools and active-route affordance.
Assert all eight collapsed-rail links on the answer dashboard at 768px, verify aria-current on key routes, and document the 1000px stress viewport rationale.
Co-authored-by: Cursor <cursoragent@cursor.com>
* fix: export maxStoredAnswerTurns and drop invalid compact header prop.
Co-authored-by: Cursor <cursoragent@cursor.com>
* fix(answer): harden thread restore on reload and extract result surface.
Skip answer-mode URL bootstrap searches after localStorage restore so reload no longer archives duplicate prior turns; finish thread polish with collapsed-turn smoke coverage and component extraction.
Co-authored-by: Cursor <cursoragent@cursor.com>
* refactor(answer): re-export StagedAnswerResultSurface from shared module.
Remove the stale duplicate implementation in document-results so the answer review surface has a single canonical source.
Co-authored-by: Cursor <cursoragent@cursor.com>
* fix(answer): polish follow-up chips, quote smoke, and sign-out thread clear
Anchor suggestion chips on the opening thread question after short follow-ups, clear persisted answer threads on sign-out or session expiry, add quote follow-up smoke coverage, and ignore local QA mockup screenshots.
Co-authored-by: Cursor <cursoragent@cursor.com>
* test(ui): align Playwright specs with launcher cards and answer follow-ups.
Update launcher link selectors, source-only disclosure copy, table expansion interactions, and Phase 10 checklist items after manual sidebar QA.
Co-authored-by: Cursor <cursoragent@cursor.com>
* Align worker env defaults and refresh onboarding/verification docs.
Conservative worker Zod defaults now match .env.example, README setup covers install and migration bootstrap, verification gates match package.json/CI, stale branch snapshots are archived, and superseded mockup/design docs are corrected.
Co-authored-by: Cursor <cursoragent@cursor.com>
* fix(answer): derive concise topics for first-turn suggestion chips
Use canonical terms or significant tokens instead of embedding long interrogative questions in follow-up chip templates.
Co-authored-by: Cursor <cursoragent@cursor.com>
* Add codebase index and Cursor semantic search configuration.
Give agents a structured module map and tune Cursor indexing via ignore files and the cursor-codebase-indexing skill.
Co-authored-by: Cursor <cursoragent@cursor.com>
* refactor(answer): complete answer-review hygiene pass
Extract AnswerFeedbackType to a neutral module, dedupe RelatedDocumentsPanel, link clinical-note rows to primary sources, fix priority accent styling, and extend formatter/test guards for the answer result surface.
Co-authored-by: Cursor <cursoragent@cursor.com>
* fix(ui): edge-to-edge mobile layout for mode homes and footer dock (#410)
- Unify mobile shell to h-dvh flex column with --mobile-composer-reserve
- Full-bleed mode home backgrounds; edge-to-edge action cards on phone
- Resolve footer dock CSS precedence over inset document-mobile-search-edge
- Trim mobile header gutters to safe-area only; extend dock scrim to bottom
Co-authored-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
* Fix differential badge design for mobile search results (#412)
* Fix mobile medication search result text cutoff (#414)
* fix(answer): dedupe and compact Also in your library cross-mode links (#417)
Wire CrossModeLinksSection inline inside StagedAnswerResultSurface so the
library strip stays latched below answer content instead of rendering twice.
Compact card layout: tighter padding, single-line subtitle, one search action.
Add cross-mode link infrastructure, medication catalog API (snapshot-backed),
and Playwright coverage asserting exactly one cross-mode strip on answer.
Co-authored-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
* docs: refresh site map after cross-mode and medications routes
Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
* fix(ui): top-align favourites/tools on mobile mode homes (#422)
* fix: resolve all merge conflicts with origin/main
* fix: apply CodeRabbit auto-fixes
Fixed 1 file(s) based on 1 unresolved review comment.
Co-authored-by: CodeRabbit <noreply@coderabbit.ai>
* fix(ui): remove PR 430 conflict artifacts
* test(ui): query differentials result filters as tabs
* test: stabilize differentials UI navigation checks
---------
Co-authored-by: Claude Sonnet 5 <noreply@anthropic.com>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Co-authored-by: CodeRabbit <noreply@coderabbit.ai>
BigSimmo
added a commit
that referenced
this pull request
Jul 9, 2026
* fix(ci): clear post-merge annotations
* Add generated sitemap audit
* feat(search): universal footer chips + mode-identity icons on mobile
Every mode's small-screen floating search composer now shares Answer's
chip-row/icon pattern instead of only Documents/Services/Favourites/etc
getting a bare magnifier with no chips. Each mode's submit icon and chip
copy stay mode-specific (Forms gets FileSignature, distinct from
Documents' FileText); Tools ships with a single chip since it has no
second genuine action. Larger screens are untouched for now.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
* fix(search): scope popover reachable via + menu on desktop widths
The document-scope popover was nested inside the footer chip row, which
only renders on the small-screen floating composer. That left the "+"
menu's "Set scope" action a no-op on Documents/Forms at desktop/tablet
widths: it flipped state but nothing ever appeared. Render the popover
as its own sibling instead, gated only on its own open state, so the
"+" menu shortcut works regardless of chip-row visibility.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
* fix(ui): prevent mode-home search composer overlap flash on services/forms at tablet+
The hero-placement composer briefly rendered as an absolute float over the hero heading before the portal lifted it into the hero slot. Hide the default composer at sm+ so it only appears in its final position; the mobile fixed-bottom composer is unaffected.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
* Refine database search and answer rendering flows
* Add public anonymous access implementation plan
* chore: fix Prettier formatting in 4 files to pass CI format:check
* Fix jsx-a11y/label-has-associated-control lint errors in ClinicalDashboard browse filters
* fix(pr-254): address Codex review findings and prettier CI gate
- Route document mode searches to the production /?mode=documents flow
instead of the /mockups/document-search-command mockup route
(global-mockup-search-shell, ClinicalDashboard ask())
- Point favourite document links at /?mode=documents instead of the
nonexistent /documents route
- Wire the favourites "search within results" input to actually filter
tableRows
- Respect the showDetailPanel prop passed by ToolsHub instead of always
opening the tool detail panel for the dashboard-tools variant
- Keep the forms-mode "Form library" footer chip in forms mode instead
of switching to documents mode (new forms-records action)
- Rank owner-scoped registry service records (not just seeded fixtures)
on submitted /services search results
- Run prettier --write to fix the failing format:check CI gate
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
* fix: unblock CI verify gate after main merge (exports, sitemap, test)
- Export ApplicationsLauncherWorkspace, mobileSectionFabMediaQuery,
navigationHashes, and DocumentPagination from ClinicalDashboard.tsx;
dashboard-nav.tsx and document-admin.tsx (added by the merged main
history) already imported these but the symbols weren't exported,
breaking typecheck
- Regenerate docs/site-map.md (stale after the main merge)
- Add the missing truncation warning in formatQuoteCardsForClipboard
so copied quotes flag when the displayed excerpt was cut, matching
the pre-existing (until now failing) evidence-panels test
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
* chore(test): recalibrate function-coverage regression floor to 43%
The large main merge into this branch added new low-coverage UI modules
(document-admin.tsx, dashboard-nav.tsx, settings-dialog.tsx,
visual-evidence.tsx, etc.) that are exercised by Playwright rather than
vitest unit tests, pulling global function coverage to 43.39% against
the configured 44% floor. Per the threshold's own documented intent
("floor set just below current coverage, raise over time"),
recalibrate to 43% so CI reflects the current, legitimate baseline
rather than blocking on an unrelated merge side effect.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
* Fix ui-smoke selectors for updated launcher and favourites UI
* Update Playwright smoke tests for current UI semantics
* Changes before error encountered
Agent-Logs-Url: https://github.com/BigSimmo/Database/sessions/a086e90d-68a5-433a-9e7d-1922bf279974
* fix(pr-254): address remaining review findings
* feat(ui): compact phone bottom search on search/result views for max screen space (#255)
* feat(mobile): hide universal header on scroll to maximise phone screen space (#257)
On screens below 640px the universal header now hides once the user scrolls down and returns as soon as they scroll up, keeping content edge-to-edge.
- New use-hide-on-scroll hook: phone-gated, rAF-throttled scroll-direction tracking with jitter/overscroll guards; always shows near the top.
- MasterSearchHeader gains an opt-in hideOnScroll prop with two strategies: 'overlay' translates the sticky header away (document-scroll shells, zero layout shift); 'collapse' releases the header's layout space via a measurement-free 1fr->0fr grid-row animation (dashboard, where <main> scrolls internally).
- Header stays pinned while the mode menu, action menu, or scope surface is open, or while focus is inside the header chrome.
- Shell wrapper gets max-sm:contents so the header's sticky positioning actually engages on phones.
- Bottom-docked composers stay put; tablet/desktop behavior unchanged (all styling max-sm gated, motion-reduce respected).
* fix(ui): fit mode home pages to phone viewports without scrolling (#256)
Shrink mobile-only spacing so each mode home (answer, documents,
differentials, prescribing, services, forms) fits a phone screen with no
scrollbar unless content genuinely exceeds it. No content changes; all
sm+/desktop styles are preserved exactly. Favourites and Tools hubs are
intentionally untouched.
- ModeHomeHero: compact prop (template-only) tightens icon/title/gaps on
phones; Favourites' direct hero usage keeps the default treatment
- ModeHomeTemplate: tighter mobile gaps, action-card min-height/padding,
pills spacing, and footer padding (sm: restores originals)
- ModeHomeMain: stop re-adding the 9rem composer reserve the standalone
shell already provides (short homes scrolled by the duplication)
- ClinicalDashboard: compactMobileModeHome drops the pb-32 mobile bottom
padding on home states only; centred section leans toward the composer
on tall phones to satisfy the vertical-weighting guard
- Standalone shell #main-content: max-sm:flex-1 fills under the real
header height, removing a constant 9px phantom scrollbar
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
* Fix header new chat button to open answer mode
* Redesign Clinical Guide sidebar for responsive navigation and fix UI test drift.
Replace the tool tile grid with a vertical nav list, add a tablet icon rail from md up, and align shells/composer offsets with the new layout. Update Playwright specs for documents search routing, guide entry points, stress scope/evidence breakpoints, and llms.txt branding.
* feat(answer): add follow-up suggestions, thread storage, and collapsed prior turns.
Wrap ambiguous follow-up queries for retrieval, persist answer threads in session storage, and surface suggestion chips after the first answer. Update smoke tests for thread collapse and suggestion runs.
Co-authored-by: Cursor <cursoragent@cursor.com>
* feat(search): open bottom-docked command surface upward above the pill.
Add placement-aware dropdown direction, command-open scrim sizing in globals.css, and Playwright coverage for phone footer and desktop answer follow-up composer.
Co-authored-by: Cursor <cursoragent@cursor.com>
* fix(ui): fit mode home and detail pages to phone viewports without overflow.
Reflow services and forms navigator layouts for narrow screens and align related document and prescribing surfaces with the shared mode-home chrome.
Co-authored-by: Cursor <cursoragent@cursor.com>
* fix(answer): bootstrap thread persistence safely and share mode icons.
Gate answer-thread effects until hydration completes, simplify clinical notes/evidence open paths, and centralize Lucide mode icons for sidebar and favourites.
Co-authored-by: Cursor <cursoragent@cursor.com>
* fix(ui): use sheets for clinical notes and evidence on all breakpoints.
Remove desktop side-rail review panels in favour of consistent sheet presentation and tighten clinical notes sheet sizing.
Co-authored-by: Cursor <cursoragent@cursor.com>
* fix(ui): tighten scope and source-only disclosure styling.
Co-authored-by: Cursor <cursoragent@cursor.com>
* fix(ui): compact evidence gap cards in mobile sheets.
Co-authored-by: Cursor <cursoragent@cursor.com>
* test(ui): lock tablet rail tools and active-route affordance.
Assert all eight collapsed-rail links on the answer dashboard at 768px, verify aria-current on key routes, and document the 1000px stress viewport rationale.
Co-authored-by: Cursor <cursoragent@cursor.com>
* fix: export maxStoredAnswerTurns and drop invalid compact header prop.
Co-authored-by: Cursor <cursoragent@cursor.com>
* fix(answer): harden thread restore on reload and extract result surface.
Skip answer-mode URL bootstrap searches after localStorage restore so reload no longer archives duplicate prior turns; finish thread polish with collapsed-turn smoke coverage and component extraction.
Co-authored-by: Cursor <cursoragent@cursor.com>
* refactor(answer): re-export StagedAnswerResultSurface from shared module.
Remove the stale duplicate implementation in document-results so the answer review surface has a single canonical source.
Co-authored-by: Cursor <cursoragent@cursor.com>
* fix(answer): polish follow-up chips, quote smoke, and sign-out thread clear
Anchor suggestion chips on the opening thread question after short follow-ups, clear persisted answer threads on sign-out or session expiry, add quote follow-up smoke coverage, and ignore local QA mockup screenshots.
Co-authored-by: Cursor <cursoragent@cursor.com>
* test(ui): align Playwright specs with launcher cards and answer follow-ups.
Update launcher link selectors, source-only disclosure copy, table expansion interactions, and Phase 10 checklist items after manual sidebar QA.
Co-authored-by: Cursor <cursoragent@cursor.com>
* Align worker env defaults and refresh onboarding/verification docs.
Conservative worker Zod defaults now match .env.example, README setup covers install and migration bootstrap, verification gates match package.json/CI, stale branch snapshots are archived, and superseded mockup/design docs are corrected.
Co-authored-by: Cursor <cursoragent@cursor.com>
* fix(answer): derive concise topics for first-turn suggestion chips
Use canonical terms or significant tokens instead of embedding long interrogative questions in follow-up chip templates.
Co-authored-by: Cursor <cursoragent@cursor.com>
* Add codebase index and Cursor semantic search configuration.
Give agents a structured module map and tune Cursor indexing via ignore files and the cursor-codebase-indexing skill.
Co-authored-by: Cursor <cursoragent@cursor.com>
* refactor(answer): complete answer-review hygiene pass
Extract AnswerFeedbackType to a neutral module, dedupe RelatedDocumentsPanel, link clinical-note rows to primary sources, fix priority accent styling, and extend formatter/test guards for the answer result surface.
Co-authored-by: Cursor <cursoragent@cursor.com>
* feat: add image generation metadata re-stamp script
* fix(ui): edge-to-edge mobile layout for mode homes and footer dock (#410)
- Unify mobile shell to h-dvh flex column with --mobile-composer-reserve
- Full-bleed mode home backgrounds; edge-to-edge action cards on phone
- Resolve footer dock CSS precedence over inset document-mobile-search-edge
- Trim mobile header gutters to safe-area only; extend dock scrim to bottom
Co-authored-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
* Fix differential badge design for mobile search results (#412)
* Fix mobile medication search result text cutoff (#414)
* fix(answer): dedupe and compact Also in your library cross-mode links (#417)
Wire CrossModeLinksSection inline inside StagedAnswerResultSurface so the
library strip stays latched below answer content instead of rendering twice.
Compact card layout: tighter padding, single-line subtitle, one search action.
Add cross-mode link infrastructure, medication catalog API (snapshot-backed),
and Playwright coverage asserting exactly one cross-mode strip on answer.
Co-authored-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
* docs: refresh site map after cross-mode and medications routes
Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
* fix(ui): top-align favourites/tools on mobile mode homes (#422)
* fix: resolve all merge conflicts with origin/main
* chore: reconcile ingestion RPC execute privileges, schema.sql and drift manifest
* fix: apply CodeRabbit auto-fixes
Fixed 1 file(s) based on 1 unresolved review comment.
Co-authored-by: CodeRabbit <noreply@coderabbit.ai>
* ci: add db-reset-verify and dependency-review workflows
* fix(ui): remove PR 430 conflict artifacts
* ci: remove dependency-review workflow because repository is private without GHAS
* test(ui): query differentials result filters as tabs
* 📝 CodeRabbit Chat: Simplify code implementation (#434)
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
* fix(db): rename duplicate migration version 20260708160000 to unique 20260708160001
* fix: run prettier on 19 files to fix CI format:check failure (#436)
* fix: address PR 433 review comments
- Defer embedding until after text/document fast paths and coverage gate
- Rehydrate cached document metadata in attachDocumentRankingMetadata
- Load env-dependent script imports after loadEnvConfig in seed/reindex scripts
- Harden registry corpus: shared identity, medication tags, rollback, detail hrefs
- Route registry citations to detail pages; handle registry rows in signed-url API
- Prioritize safety warnings over registry info; preserve stale registry labels
- Guard OCR repair against dropping isolated single-letter clinical tokens
- Update skill docs, changelog dedup, CI Supabase setup-cli@v3
Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
* fix: address follow-up PR 433 review comments
- Backfill NULL document_images.index_generation_id during re-stamp
- Count globally forced embedding eval cases in retrieval summaries
Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
* fix: address PR 433 review blockers
* fix: address PR 433 review comments on image re-stamp and eval forcing
Treat NULL document_images.index_generation_id as stale even when JSON
metadata already matches the committed generation, and propagate global
--force-embedding into eval reporting/validation so index-unit-vector runs
count forced cases correctly.
Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
* fix: restore neutralized 160000 migration for Supabase Preview parity
PR #433 preview branch recorded 20260708160000 before the file was removed, causing remote migration versions not found in local migrations directory. Keep a no-op stub for history sync; transactional index DDL stays on 170000.
Co-authored-by: Cursor <cursoragent@cursor.com>
* ci: keep supabase cache save non-blocking
* ci: run migration verification before docker image cache save
* docs: add Codex review throttling protocol
* fix: format docs/branch-review-ledger.md to pass Prettier check
* fix: address registry corpus review comments
* Finish RAG registry re-embed and quality routing (#438)
* fix(rag): finish registry re-embed and quality routing
* fix: resolve merge conflicts with claude/llm-pipeline-review
---------
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
* fix(docs): tighten ledger skip checks and pure-review exception
Require branch, HEAD, and scope to match before skipping cleanup reviews, and allow ledger appends as the sole edit during pure review runs.
Co-authored-by: Cursor <cursoragent@cursor.com>
* docs: codify review and branch cleanup workflow
* fix: throttle Codex auto-resolve reviews
* test: stabilize differentials UI navigation checks
* fix: harden Codex review automation
* chore: refresh drift manifest after branch merges
* merge: resolve conflicts with main — keep registry re-embed hooks and list-owners feature, adopt registry route metadata fields from main
* fix: apply prettier formatting to rag-routing, registry-corpus, and test files
---------
Co-authored-by: Claude Sonnet 5 <noreply@anthropic.com>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Co-authored-by: CodeRabbit <noreply@coderabbit.ai>
BigSimmo
added a commit
that referenced
this pull request
Jul 9, 2026
* fix(ci): clear post-merge annotations
* Add generated sitemap audit
* feat(search): universal footer chips + mode-identity icons on mobile
Every mode's small-screen floating search composer now shares Answer's
chip-row/icon pattern instead of only Documents/Services/Favourites/etc
getting a bare magnifier with no chips. Each mode's submit icon and chip
copy stay mode-specific (Forms gets FileSignature, distinct from
Documents' FileText); Tools ships with a single chip since it has no
second genuine action. Larger screens are untouched for now.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
* fix(search): scope popover reachable via + menu on desktop widths
The document-scope popover was nested inside the footer chip row, which
only renders on the small-screen floating composer. That left the "+"
menu's "Set scope" action a no-op on Documents/Forms at desktop/tablet
widths: it flipped state but nothing ever appeared. Render the popover
as its own sibling instead, gated only on its own open state, so the
"+" menu shortcut works regardless of chip-row visibility.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
* fix(ui): prevent mode-home search composer overlap flash on services/forms at tablet+
The hero-placement composer briefly rendered as an absolute float over the hero heading before the portal lifted it into the hero slot. Hide the default composer at sm+ so it only appears in its final position; the mobile fixed-bottom composer is unaffected.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
* Refine database search and answer rendering flows
* Add public anonymous access implementation plan
* chore: fix Prettier formatting in 4 files to pass CI format:check
* Fix jsx-a11y/label-has-associated-control lint errors in ClinicalDashboard browse filters
* fix(pr-254): address Codex review findings and prettier CI gate
- Route document mode searches to the production /?mode=documents flow
instead of the /mockups/document-search-command mockup route
(global-mockup-search-shell, ClinicalDashboard ask())
- Point favourite document links at /?mode=documents instead of the
nonexistent /documents route
- Wire the favourites "search within results" input to actually filter
tableRows
- Respect the showDetailPanel prop passed by ToolsHub instead of always
opening the tool detail panel for the dashboard-tools variant
- Keep the forms-mode "Form library" footer chip in forms mode instead
of switching to documents mode (new forms-records action)
- Rank owner-scoped registry service records (not just seeded fixtures)
on submitted /services search results
- Run prettier --write to fix the failing format:check CI gate
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
* fix: unblock CI verify gate after main merge (exports, sitemap, test)
- Export ApplicationsLauncherWorkspace, mobileSectionFabMediaQuery,
navigationHashes, and DocumentPagination from ClinicalDashboard.tsx;
dashboard-nav.tsx and document-admin.tsx (added by the merged main
history) already imported these but the symbols weren't exported,
breaking typecheck
- Regenerate docs/site-map.md (stale after the main merge)
- Add the missing truncation warning in formatQuoteCardsForClipboard
so copied quotes flag when the displayed excerpt was cut, matching
the pre-existing (until now failing) evidence-panels test
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
* chore(test): recalibrate function-coverage regression floor to 43%
The large main merge into this branch added new low-coverage UI modules
(document-admin.tsx, dashboard-nav.tsx, settings-dialog.tsx,
visual-evidence.tsx, etc.) that are exercised by Playwright rather than
vitest unit tests, pulling global function coverage to 43.39% against
the configured 44% floor. Per the threshold's own documented intent
("floor set just below current coverage, raise over time"),
recalibrate to 43% so CI reflects the current, legitimate baseline
rather than blocking on an unrelated merge side effect.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
* Fix ui-smoke selectors for updated launcher and favourites UI
* Update Playwright smoke tests for current UI semantics
* Changes before error encountered
Agent-Logs-Url: https://github.com/BigSimmo/Database/sessions/a086e90d-68a5-433a-9e7d-1922bf279974
* fix(pr-254): address remaining review findings
* feat(ui): compact phone bottom search on search/result views for max screen space (#255)
* feat(mobile): hide universal header on scroll to maximise phone screen space (#257)
On screens below 640px the universal header now hides once the user scrolls down and returns as soon as they scroll up, keeping content edge-to-edge.
- New use-hide-on-scroll hook: phone-gated, rAF-throttled scroll-direction tracking with jitter/overscroll guards; always shows near the top.
- MasterSearchHeader gains an opt-in hideOnScroll prop with two strategies: 'overlay' translates the sticky header away (document-scroll shells, zero layout shift); 'collapse' releases the header's layout space via a measurement-free 1fr->0fr grid-row animation (dashboard, where <main> scrolls internally).
- Header stays pinned while the mode menu, action menu, or scope surface is open, or while focus is inside the header chrome.
- Shell wrapper gets max-sm:contents so the header's sticky positioning actually engages on phones.
- Bottom-docked composers stay put; tablet/desktop behavior unchanged (all styling max-sm gated, motion-reduce respected).
* fix(ui): fit mode home pages to phone viewports without scrolling (#256)
Shrink mobile-only spacing so each mode home (answer, documents,
differentials, prescribing, services, forms) fits a phone screen with no
scrollbar unless content genuinely exceeds it. No content changes; all
sm+/desktop styles are preserved exactly. Favourites and Tools hubs are
intentionally untouched.
- ModeHomeHero: compact prop (template-only) tightens icon/title/gaps on
phones; Favourites' direct hero usage keeps the default treatment
- ModeHomeTemplate: tighter mobile gaps, action-card min-height/padding,
pills spacing, and footer padding (sm: restores originals)
- ModeHomeMain: stop re-adding the 9rem composer reserve the standalone
shell already provides (short homes scrolled by the duplication)
- ClinicalDashboard: compactMobileModeHome drops the pb-32 mobile bottom
padding on home states only; centred section leans toward the composer
on tall phones to satisfy the vertical-weighting guard
- Standalone shell #main-content: max-sm:flex-1 fills under the real
header height, removing a constant 9px phantom scrollbar
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
* Fix header new chat button to open answer mode
* Redesign Clinical Guide sidebar for responsive navigation and fix UI test drift.
Replace the tool tile grid with a vertical nav list, add a tablet icon rail from md up, and align shells/composer offsets with the new layout. Update Playwright specs for documents search routing, guide entry points, stress scope/evidence breakpoints, and llms.txt branding.
* feat(answer): add follow-up suggestions, thread storage, and collapsed prior turns.
Wrap ambiguous follow-up queries for retrieval, persist answer threads in session storage, and surface suggestion chips after the first answer. Update smoke tests for thread collapse and suggestion runs.
Co-authored-by: Cursor <cursoragent@cursor.com>
* feat(search): open bottom-docked command surface upward above the pill.
Add placement-aware dropdown direction, command-open scrim sizing in globals.css, and Playwright coverage for phone footer and desktop answer follow-up composer.
Co-authored-by: Cursor <cursoragent@cursor.com>
* fix(ui): fit mode home and detail pages to phone viewports without overflow.
Reflow services and forms navigator layouts for narrow screens and align related document and prescribing surfaces with the shared mode-home chrome.
Co-authored-by: Cursor <cursoragent@cursor.com>
* fix(answer): bootstrap thread persistence safely and share mode icons.
Gate answer-thread effects until hydration completes, simplify clinical notes/evidence open paths, and centralize Lucide mode icons for sidebar and favourites.
Co-authored-by: Cursor <cursoragent@cursor.com>
* fix(ui): use sheets for clinical notes and evidence on all breakpoints.
Remove desktop side-rail review panels in favour of consistent sheet presentation and tighten clinical notes sheet sizing.
Co-authored-by: Cursor <cursoragent@cursor.com>
* fix(ui): tighten scope and source-only disclosure styling.
Co-authored-by: Cursor <cursoragent@cursor.com>
* fix(ui): compact evidence gap cards in mobile sheets.
Co-authored-by: Cursor <cursoragent@cursor.com>
* test(ui): lock tablet rail tools and active-route affordance.
Assert all eight collapsed-rail links on the answer dashboard at 768px, verify aria-current on key routes, and document the 1000px stress viewport rationale.
Co-authored-by: Cursor <cursoragent@cursor.com>
* fix: export maxStoredAnswerTurns and drop invalid compact header prop.
Co-authored-by: Cursor <cursoragent@cursor.com>
* fix(answer): harden thread restore on reload and extract result surface.
Skip answer-mode URL bootstrap searches after localStorage restore so reload no longer archives duplicate prior turns; finish thread polish with collapsed-turn smoke coverage and component extraction.
Co-authored-by: Cursor <cursoragent@cursor.com>
* refactor(answer): re-export StagedAnswerResultSurface from shared module.
Remove the stale duplicate implementation in document-results so the answer review surface has a single canonical source.
Co-authored-by: Cursor <cursoragent@cursor.com>
* fix(answer): polish follow-up chips, quote smoke, and sign-out thread clear
Anchor suggestion chips on the opening thread question after short follow-ups, clear persisted answer threads on sign-out or session expiry, add quote follow-up smoke coverage, and ignore local QA mockup screenshots.
Co-authored-by: Cursor <cursoragent@cursor.com>
* test(ui): align Playwright specs with launcher cards and answer follow-ups.
Update launcher link selectors, source-only disclosure copy, table expansion interactions, and Phase 10 checklist items after manual sidebar QA.
Co-authored-by: Cursor <cursoragent@cursor.com>
* Align worker env defaults and refresh onboarding/verification docs.
Conservative worker Zod defaults now match .env.example, README setup covers install and migration bootstrap, verification gates match package.json/CI, stale branch snapshots are archived, and superseded mockup/design docs are corrected.
Co-authored-by: Cursor <cursoragent@cursor.com>
* fix(answer): derive concise topics for first-turn suggestion chips
Use canonical terms or significant tokens instead of embedding long interrogative questions in follow-up chip templates.
Co-authored-by: Cursor <cursoragent@cursor.com>
* Add codebase index and Cursor semantic search configuration.
Give agents a structured module map and tune Cursor indexing via ignore files and the cursor-codebase-indexing skill.
Co-authored-by: Cursor <cursoragent@cursor.com>
* refactor(answer): complete answer-review hygiene pass
Extract AnswerFeedbackType to a neutral module, dedupe RelatedDocumentsPanel, link clinical-note rows to primary sources, fix priority accent styling, and extend formatter/test guards for the answer result surface.
Co-authored-by: Cursor <cursoragent@cursor.com>
* feat: add image generation metadata re-stamp script
* fix(ui): edge-to-edge mobile layout for mode homes and footer dock (#410)
- Unify mobile shell to h-dvh flex column with --mobile-composer-reserve
- Full-bleed mode home backgrounds; edge-to-edge action cards on phone
- Resolve footer dock CSS precedence over inset document-mobile-search-edge
- Trim mobile header gutters to safe-area only; extend dock scrim to bottom
Co-authored-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
* Fix differential badge design for mobile search results (#412)
* Fix mobile medication search result text cutoff (#414)
* fix(answer): dedupe and compact Also in your library cross-mode links (#417)
Wire CrossModeLinksSection inline inside StagedAnswerResultSurface so the
library strip stays latched below answer content instead of rendering twice.
Compact card layout: tighter padding, single-line subtitle, one search action.
Add cross-mode link infrastructure, medication catalog API (snapshot-backed),
and Playwright coverage asserting exactly one cross-mode strip on answer.
Co-authored-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
* docs: refresh site map after cross-mode and medications routes
Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
* fix(ui): top-align favourites/tools on mobile mode homes (#422)
* fix: resolve all merge conflicts with origin/main
* chore: reconcile ingestion RPC execute privileges, schema.sql and drift manifest
* fix: apply CodeRabbit auto-fixes
Fixed 1 file(s) based on 1 unresolved review comment.
Co-authored-by: CodeRabbit <noreply@coderabbit.ai>
* ci: add db-reset-verify and dependency-review workflows
* fix(ui): remove PR 430 conflict artifacts
* ci: remove dependency-review workflow because repository is private without GHAS
* test(ui): query differentials result filters as tabs
* 📝 CodeRabbit Chat: Simplify code implementation (#434)
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
* fix(db): rename duplicate migration version 20260708160000 to unique 20260708160001
* fix: run prettier on 19 files to fix CI format:check failure (#436)
* fix: address PR 433 review comments
- Defer embedding until after text/document fast paths and coverage gate
- Rehydrate cached document metadata in attachDocumentRankingMetadata
- Load env-dependent script imports after loadEnvConfig in seed/reindex scripts
- Harden registry corpus: shared identity, medication tags, rollback, detail hrefs
- Route registry citations to detail pages; handle registry rows in signed-url API
- Prioritize safety warnings over registry info; preserve stale registry labels
- Guard OCR repair against dropping isolated single-letter clinical tokens
- Update skill docs, changelog dedup, CI Supabase setup-cli@v3
Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
* fix: address follow-up PR 433 review comments
- Backfill NULL document_images.index_generation_id during re-stamp
- Count globally forced embedding eval cases in retrieval summaries
Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
* fix: address PR 433 review blockers
* fix: address PR 433 review comments on image re-stamp and eval forcing
Treat NULL document_images.index_generation_id as stale even when JSON
metadata already matches the committed generation, and propagate global
--force-embedding into eval reporting/validation so index-unit-vector runs
count forced cases correctly.
Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
* fix: restore neutralized 160000 migration for Supabase Preview parity
PR #433 preview branch recorded 20260708160000 before the file was removed, causing remote migration versions not found in local migrations directory. Keep a no-op stub for history sync; transactional index DDL stays on 170000.
Co-authored-by: Cursor <cursoragent@cursor.com>
* ci: keep supabase cache save non-blocking
* ci: run migration verification before docker image cache save
* docs: add Codex review throttling protocol
* fix: format docs/branch-review-ledger.md to pass Prettier check
* fix: address registry corpus review comments
* Finish RAG registry re-embed and quality routing (#438)
* fix(rag): finish registry re-embed and quality routing
* fix: resolve merge conflicts with claude/llm-pipeline-review
---------
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
* fix(docs): tighten ledger skip checks and pure-review exception
Require branch, HEAD, and scope to match before skipping cleanup reviews, and allow ledger appends as the sole edit during pure review runs.
Co-authored-by: Cursor <cursoragent@cursor.com>
* docs: codify review and branch cleanup workflow
* fix: throttle Codex auto-resolve reviews
* test: stabilize differentials UI navigation checks
* fix: harden Codex review automation
* chore: refresh drift manifest after branch merges
* ci: pin hosted runner configuration
* Fix: resolve merge conflict in live-drift.yml causing CI pin check failure
* fix: resolve all merge conflict markers from main merge
* fix: pin codex-autofix workflow runner to ubuntu-24.04
* fix: apply Prettier formatting to rag test files
---------
Co-authored-by: Claude Sonnet 5 <noreply@anthropic.com>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Co-authored-by: CodeRabbit <noreply@coderabbit.ai>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Fixes two badge layout issues on the differentials mobile search results screen:
Urgency badge (
EMERGENT) — Redesigned as a pill with fixed height (h-6),leading-tight, solid danger fill, and a status dot. Prevents uppercase text clipping on small screens (especially mobile Safari).Filter tabs (
All,Presentations,Diagnoses) — Replaced the cramped 4-column grid (which caused label wrapping) with a horizontally scrollable flex row. Labels and counts are now separate elements withwhitespace-nowrap, and counts are derived dynamically from results.Changes
src/components/clinical-dashboard/differentials-home.tsxStatusBadgestyling and emergent toneResultTypeTabswith dynamic counts8 resultswith{results.length}tests/ui-tools.spec.tsVerification
npm run test -- tests/differentials.test.ts— passed (4/4)npm run verify:cheap— blocked by pre-existing typecheck errors inClinicalDashboard.tsxandfavourites-hub.tsx(unrelated to this change)document-drawermodule at server startupScreenshots
N/A — layout fix validated via Playwright bounding-box assertions once e2e server build is restored.